Skip to content

feat: forgejoのパブリックリポジトリを使えるようにする#1296

Merged
ncaq merged 1 commit into
masterfrom
forgejo-usable-public-repo
Jun 27, 2026
Merged

feat: forgejoのパブリックリポジトリを使えるようにする#1296
ncaq merged 1 commit into
masterfrom
forgejo-usable-public-repo

Conversation

@ncaq

@ncaq ncaq commented Jun 27, 2026

Copy link
Copy Markdown
Owner

Git LFSを使う上でサインインしないと全く使えない場合、
GitHubなどのパブリックリポジトリでGit LFSが当然使えなくなるため、
サインインしなくても見ることはできるようにします。
プライベートリポジトリはそれぞれの権限で守られますし、
アカウント作成も許可していないので、
権限なしでプライベートな空間にアクセスされることはありません。

Git LFSを使う上でサインインしないと全く使えない場合、
GitHubなどのパブリックリポジトリでGit LFSが当然使えなくなるため、
サインインしなくても見ることはできるようにします。
プライベートリポジトリはそれぞれの権限で守られますし、
アカウント作成も許可していないので、
権限なしでプライベートな空間にアクセスされることはありません。
@ncaq ncaq self-assigned this Jun 27, 2026
@ncaq ncaq added the Type: Feature New Feature label Jun 27, 2026
@github-actions

Copy link
Copy Markdown

nvd diff: seminar (base: master)

Version changes

Closure size: 4975 -> 4975 (13 paths added, 13 paths removed, delta +0, disk usage -24B).

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

概要

REQUIRE_SIGNIN_VIEW = true を削除し、未認証ユーザーが公開リポジトリを閲覧できるようにする変更です。Git LFSを公開リポジトリで使えるようにするという目的に沿っており、DISABLE_REGISTRATION = true が維持されているため新規アカウント登録は引き続き不可で、未認証閲覧から認証操作への権限昇格リスクは限定的です。意図的かつ妥当な変更と判断します。

コード品質上の新たな問題はありません。一方で公開範囲を広げる変更のため、防御的な設定の追加と意図を残すコメントの追加を提案します。

Review metadata
  • Reviewed commit: 3848ccc
  • PR: #1296
  • kyosei: 3.5.0
  • kyosei-action: 2.2.5
  • Claude Code: 2.1.174
  • Model: claude-opus-4-8
  • Execution: GitHub Actions (run)

Comment thread nixos/host/seminar/forgejo.nix
Comment thread nixos/host/seminar/forgejo.nix
Comment thread nixos/host/seminar/forgejo.nix
@ncaq ncaq merged commit 14015f2 into master Jun 27, 2026
10 checks passed
@ncaq ncaq deleted the forgejo-usable-public-repo branch June 27, 2026 04:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Feature New Feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant